Skip to content

Fix Major upgrade article for 2.7.1 changes #5394

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 8, 2015

Conversation

wouterj
Copy link
Member

@wouterj wouterj commented Jun 13, 2015

As of 2.7.1, deprecation errors are silenced, causing a lot less trouble.

This PR also adds some information about the SYMFONY_DEPRECATIONS_HELPER env variable.

Q A
Doc fix? yes
New docs? yes
Applies to all
Fixed tickets -


To make sure this doesn't happen, you can install the PHPUnit bridge:
If you want to make sure you don't use any deprecated features, you can install
the Symfony PHPunit bridge:

.. code-block:: bash

$ composer require symfony/phpunit-bridge
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for commenting so late in a pull request labeled as finished, but I think this command should be:

$ composer require --dev symfony/phpunit-bridge

@wouterj
Copy link
Member Author

wouterj commented Jul 28, 2015

Applied the suggestions of @javiereguiluz and did some updates as #5413 was merged. Ready to merge imo

@xabbuh
Copy link
Member

xabbuh commented Jul 28, 2015

👍

@weaverryan weaverryan merged commit 9d6db3e into symfony:2.3 Aug 8, 2015
weaverryan added a commit that referenced this pull request Aug 8, 2015
This PR was merged into the 2.3 branch.

Discussion
----------

Fix Major upgrade article for 2.7.1 changes

As of 2.7.1, deprecation errors are silenced, causing a lot less trouble.

This PR also adds some information about the `SYMFONY_DEPRECATIONS_HELPER` env variable.

| Q | A
| --- | ---
| Doc fix? | yes
| New docs? | yes
| Applies to | all
| Fixed tickets | -

Commits
-------

9d6db3e Fix Major upgrade article for 2.7.1 changes
@wouterj wouterj deleted the update_2.7.1 branch August 8, 2015 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants